Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Abstract (start, end) from Spanning struct into new struct Span #1207

Closed
wants to merge 2 commits into from

Conversation

audunhalland
Copy link
Contributor

This makes it easier to treat the source region (the Span) separately from the spanned item in Spanning.

@audunhalland audunhalland mentioned this pull request Nov 8, 2023
@tyranron tyranron added enhancement Improvement of existing features or bugfix semver::breaking Breaking change in terms of SemVer k::api Related to API (application interface) k::refactor Refactoring, technical debt elimination and other improvements of existing code base labels Nov 9, 2023
Copy link
Member

@tyranron tyranron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@audunhalland please, make this PR editable for maintainers:
Screenshot 2023-11-09 at 14 29 45

@tyranron tyranron added this to the 0.16.0 milestone Nov 9, 2023
@audunhalland
Copy link
Contributor Author

The checkbox isn't visible for me. Could it be because the fork is owned by an organization in which I'm not an administrator? 🤔

Perhaps it's better if I re-create the PR from my personal fork.

@audunhalland
Copy link
Contributor Author

New PR up at #1208 (from a my personal fork). Sorry for the noise 😿

The "Allow edits" github bug is discussed here: https://github.com/orgs/community/discussions/5634

@tyranron tyranron added duplicate This issue or pull request already exists wontfix This will not be worked on labels Nov 9, 2023
@tyranron
Copy link
Member

tyranron commented Nov 9, 2023

@audunhalland thanks!

tyranron pushed a commit that referenced this pull request Nov 9, 2023
@tyranron tyranron added the area::parsing Related to GraphQL parsing label Nov 9, 2023
@audunhalland audunhalland deleted the spanning-span branch November 9, 2023 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area::parsing Related to GraphQL parsing duplicate This issue or pull request already exists enhancement Improvement of existing features or bugfix k::api Related to API (application interface) k::refactor Refactoring, technical debt elimination and other improvements of existing code base semver::breaking Breaking change in terms of SemVer wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants